Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport 2.3] 64: Added rounding when using aggreagate script for avg metric. Added… #553

Closed
wants to merge 1 commit into from

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport fadc553 from #490

#490)

* 64: Added rounding when using aggreagate script for avg metric. Added unit tests for checking average aggregations against the target rollup index

Signed-off-by: Stevan Buzejic <buzejic.stevan@gmail.com>

* 64: Rollup job renamed

Signed-off-by: Stevan Buzejic <buzejic.stevan@gmail.com>

* 64: Removed unrelevant metrics for the avg calculation test

Signed-off-by: Stevan Buzejic <buzejic.stevan@gmail.com>

Signed-off-by: Stevan Buzejic <buzejic.stevan@gmail.com>
(cherry picked from commit fadc553)
@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team October 6, 2022 04:17
@codecov-commenter
Copy link

Codecov Report

Merging #553 (eaeb9a1) into 2.3 (9af3b15) will increase coverage by 0.06%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                2.3     #553      +/-   ##
============================================
+ Coverage     75.79%   75.85%   +0.06%     
- Complexity     2476     2477       +1     
============================================
  Files           315      315              
  Lines         14511    14511              
  Branches       2245     2245              
============================================
+ Hits          10999    11008       +9     
+ Misses         2255     2252       -3     
+ Partials       1257     1251       -6     
Impacted Files Coverage Δ
...nsearch/indexmanagement/rollup/util/RollupUtils.kt 80.83% <100.00%> (ø)
...nt/indexstatemanagement/model/destination/Slack.kt 55.55% <0.00%> (-22.23%) ⬇️
.../rollup/action/start/TransportStartRollupAction.kt 71.76% <0.00%> (-3.53%) ⬇️
.../action/explain/TransportExplainTransformAction.kt 70.78% <0.00%> (-2.25%) ⬇️
...ent/indexstatemanagement/util/ManagedIndexUtils.kt 76.10% <0.00%> (-1.77%) ⬇️
...gement/indexstatemanagement/action/ShrinkAction.kt 69.11% <0.00%> (-1.48%) ⬇️
...nt/rollup/action/stop/TransportStopRollupAction.kt 76.47% <0.00%> (-1.18%) ⬇️
...nt/indexstatemanagement/ManagedIndexCoordinator.kt 68.71% <0.00%> (-0.59%) ⬇️
...earch/indexmanagement/transform/model/Transform.kt 85.04% <0.00%> (-0.43%) ⬇️
.../opensearch/indexmanagement/rollup/model/Rollup.kt 86.04% <0.00%> (+0.46%) ⬆️
... and 5 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants